appDisplay: don't increment counter if we didn't add a category

Or we'll later try to jump to the wrong category when clicking on it.

diff --git a/js/ui/appDisplay.js b/js/ui/appDisplay.js
index 300c7e75a722c94cc64aff5c2c24ca6bbdb91c95..851a5118d4ef0e7cdf9c8430c3826011ce90b5cd 100644 (file)
--- a/js/ui/appDisplay.js
+++ b/js/ui/appDisplay.js
@@ -223,7 +223,7 @@ const ViewByCategories = new Lang.Class({
             this._loadCategory(dir, apps);
 
             if (apps.length == 0)
-                return;
+                return false;
         }
 
         let button = new St.Button({ label: GLib.markup_escape_text (name, -1),
@@ -244,6 +244,7 @@ const ViewByCategories = new Lang.Class({
         }
 
         this._categoryBox.add(button, { expand: true, x_fill: true, y_fill: false });
+        return true;
     },
 
     _removeAll: function() {
@@ -269,8 +270,9 @@ const ViewByCategories = new Lang.Class({
                 var dir = iter.get_directory();
                 if (dir.get_is_nodisplay())
                     continue;
-                this._addCategory(dir.get_name(), i, dir);
-                i++;
+
+                if (this._addCategory(dir.get_name(), i, dir))
+                    i++;
             }
         }